-
Notifications
You must be signed in to change notification settings - Fork 3
feat(create): cleaner client template #181
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
🚀 Package Preview Available!Install this PR's preview build with npm: npm i @base44-preview/cli@0.0.30-pr.181.ddbe8c1Prefer not to change any import paths? Install using npm alias so your code still imports npm i "base44@npm:@base44-preview/cli@0.0.30-pr.181.ddbe8c1"Or add it to your {
"dependencies": {
"base44": "npm:@base44-preview/cli@0.0.30-pr.181.ddbe8c1"
}
}
Preview published to npm registry — try new features instantly! |
Code ReviewFound 1 issue that needs attention: CSS Variables Removed But Still ReferencedLocation: Issue: The PR removes all CSS custom property definitions from
Impact: Since these CSS variables no longer exist, the expressions will evaluate to invalid values like Suggested fix: Since the goal is to simplify to a minimal starter and shadcn UI components are removed, clean up both files:
Alternatively, if you want to keep the theming system, restore the Reference: See tailwind.config.js lines 7-36 for the variable references. |
Code reviewIssue: CSS variables removed but still referencedRemoving the
At runtime, these produce invalid Fix options:
Location: cli/templates/backend-and-client/src/index.css Lines 5 to 11 in 7aa5a6d
|
Note
Description
This PR simplifies the backend-and-client template by removing the pre-built task management demo app and replacing it with a minimal "Hello World" starter. This provides a cleaner starting point for new projects without opinionated example code that users would need to remove anyway.
Related Issue
None
Type of Change
Changes Made
App.jsxand replaced with minimal "Hello World" componentButton,Checkbox,Input, andBase44Logocomponents.jsonshadcn configuration filetask.jsonc,task_manager.jsonc)lucide-reactdependency frompackage.jsonindex.cssby removing shadcn/ui CSS variables (retained Tailwind base styles)Testing
npm test)Checklist
Additional Notes
This change improves the developer experience for the
backend-and-clienttemplate by providing a clean slate. Users can now start with a simple "Hello World" app and add their own components and functionality without needing to delete example code first. The template still includes all necessary Base44 setup and Tailwind CSS configuration.🤖 Generated by Claude | 2026-02-09 16:45 UTC